Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: no padding if raw clienthello is too short #263

Merged
merged 1 commit into from
Dec 11, 2023
Merged

fix: no padding if raw clienthello is too short #263

merged 1 commit into from
Dec 11, 2023

Conversation

gaukas
Copy link
Contributor

@gaukas gaukas commented Dec 11, 2023

  • Add a function AlwaysPadToLen to generate padding styles according to the raw clienthello.
  • Add an extra step in FromRaw after parsing the extension list to update the padding style.

Fix #262.

@gaukas
fix: no padding if raw clienthello is too short
76ee142 
Add a function `AlwaysPadToLen` to generate padding styles according to the raw clienthello.
Add an extra step in `FromRaw` after parsing the extension list to update the padding style.
@gaukas gaukas added the bug Unexpected behavior confirmed and should be fixed label Dec 11, 2023
@gaukas gaukas self-assigned this Dec 11, 2023
@gaukas gaukas merged commit feb5a95 into master Dec 11, 2023
6 checks passed
@gaukas gaukas deleted the fix-short-raw-fp-padding branch December 11, 2023 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@gaukas gaukas

Labels
bug Unexpected behavior confirmed and should be fixed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support for padding extension
1 participant
@gaukas